Skip to content

Add ERC7984Freezable#151

Merged
james-toussaint merged 23 commits intomasterfrom
feature/confidential-freezable-token
Aug 25, 2025
Merged

Add ERC7984Freezable#151
james-toussaint merged 23 commits intomasterfrom
feature/confidential-freezable-token

Conversation

@arr00
Copy link
Copy Markdown
Member

@arr00 arr00 commented Aug 7, 2025

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 7, 2025 •
edited
Loading

✅ Deploy Preview for confidential-tokens ready!

Name Link
🔨 Latest commit 9d8aa7d
🔍 Latest deploy log https://app.netlify.com/projects/confidential-tokens/deploys/68ac789a6f18ea00085e21f2
😎 Deploy Preview https://deploy-preview-151--confidential-tokens.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@james-toussaint james-toussaint mentioned this pull request Aug 8, 2025
Closed
@james-toussaint james-toussaint self-assigned this Aug 12, 2025
@james-toussaint james-toussaint marked this pull request as ready for review August 19, 2025 08:26
@james-toussaint james-toussaint requested a review from a team as a code owner August 19, 2025 08:26
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/mocks/token/ERC7984FreezableMock.sol
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 19, 2025
View reviewed changes
Comment thread contracts/token/extensions/ERC7984Freezable.sol Outdated
james-toussaint and others added 2 commits August 21, 2025 16:03
@james-toussaint @arr00
Apply suggestions
8e08500 
Co-authored-by: Arr00 <13561405+arr00@users.noreply.github.com>
@arr00 @james-toussaint
Remove account from _validateHandleAllowance (#178)
7e45893 
Co-authored-by: James Toussaint <33313130+james-toussaint@users.noreply.github.com>
@arr00 arr00 changed the title Confidential freezable token Add ERC7984Freezable Aug 21, 2025
@arr00
Copy link
Copy Markdown
Member Author

arr00 commented Aug 21, 2025

Need to add the extension to the docs

arr00
arr00 commented Aug 22, 2025
View reviewed changes
Comment thread .changeset/seven-books-dig.md Outdated
arr00
arr00 commented Aug 22, 2025
View reviewed changes
Comment thread contracts/mocks/token/ERC7984FreezableMock.sol Outdated
arr00
arr00 commented Aug 22, 2025
View reviewed changes
Comment thread contracts/token/ERC7984/extensions/ERC7984Freezable.sol Outdated
arr00
arr00 commented Aug 22, 2025
View reviewed changes
Comment thread contracts/token/README.adoc Outdated
@arr00
Copy link
Copy Markdown
Member Author

arr00 commented Aug 22, 2025

looking good

james-toussaint
james-toussaint reviewed Aug 25, 2025
View reviewed changes
Comment thread contracts/token/README.adoc Outdated
james-toussaint
james-toussaint reviewed Aug 25, 2025
View reviewed changes
Comment thread contracts/mocks/token/ERC7984FreezableMock.sol Outdated
@james-toussaint @arr00
Apply suggestions
7f9b646 
Co-authored-by: Arr00 <13561405+arr00@users.noreply.github.com>
arr00
arr00 commented Aug 25, 2025
View reviewed changes
Copy link
Copy Markdown
Member Author

@arr00 arr00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't approve but would if I could.

Comment thread test/token/ERC7984/extensions/ERC7984Freezable.test.ts Outdated
@james-toussaint james-toussaint enabled auto-merge (squash) August 25, 2025 14:56
james-toussaint
james-toussaint approved these changes Aug 25, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@james-toussaint james-toussaint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved here: #151 (review)

@james-toussaint james-toussaint merged commit cbbf27f into master Aug 25, 2025
14 checks passed
@james-toussaint james-toussaint deleted the feature/confidential-freezable-token branch August 25, 2025 14:57
@sourcery-ai sourcery-ai Bot mentioned this pull request Sep 5, 2025
Merged
@james-toussaint james-toussaint mentioned this pull request Sep 16, 2025
Merged
@github-actions github-actions Bot mentioned this pull request Oct 9, 2025
Merged
@github-actions github-actions Bot mentioned this pull request Nov 12, 2025
Merged
@github-actions github-actions Bot mentioned this pull request Dec 12, 2025
Merged
@github-actions github-actions Bot mentioned this pull request Mar 5, 2026
Merged
@github-actions github-actions Bot mentioned this pull request Mar 20, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@james-toussaint james-toussaint james-toussaint approved these changes

Assignees

@james-toussaint james-toussaint

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@arr00 @james-toussaint @OZ-Antonio